Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: 러닝 결과 저장 V2 API 구현(러닝 경로 추가) #324

Merged
merged 10 commits into from
Dec 25, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Dec 23, 2024

🔗 이슈 연결

🚀 구현한 API

  • POST /api/v2/running-records

💡 반영할 내용 및 변경 사항 요약

  • 러닝 결과를 저장하는 API v2를 추가합니다.
    • 챌린지 및 목표 request 형식이 달라졌습니다.
    • 러닝 경로를 저장합니다.
  • 챌린지 및 목표 성취에 대한 저장 방식이 추후 변경될 가능성이 있습니다.

🔍 리뷰 요청/참고 사항

  • 서비스단에서 가공되지 않은 러닝 결과 dto인 RunningResultDto를 추가했습니다. 러닝 결과 조회 시 해당 Dto를 사용할 예정입니다. 데이터 가공은 각 프레젠테이션 계층에서 진행됩니다.
  • 현재 기획에서는 고도가 러닝 경로에 들어가지 않아, altitude의 기본 값을 0으로 지정했습니다.
  • com.dnd.runus.presentation.v1.running.dto.request에 있는 RunningAchievementMode는 변경 가능성이 매우 작고, 다른 버전에서 공통으로 사용되며, 서비스 계층에서도 사용되어서 해당 클래스를 com.dnd.runus.application.running.dto 패키지로 옮겨서 사용하는 건 어떻게 생각하는지 궁금합니다.
    • 현재 RunningAchievementMode를 패키지 경로로 명시해 둔 부분이 있는데 이 부분은 위의 사항이 결정되면 변경할 예정입니다.
  • 해당 pr이 완료되고, 목표와 챌린지 형식을 통합하는 pr를 올릴 예정입니다.(러닝 결과 챌린지, 목표 달성 결과 관련 반환 dto 통합 #317 )

@hee9841 hee9841 added the feature 기능 추가 label Dec 23, 2024
@hee9841 hee9841 self-assigned this Dec 23, 2024
@hee9841 hee9841 requested a review from WonSteps December 23, 2024 23:34
Copy link
Member

@WonSteps WonSteps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 👍

@hee9841 hee9841 merged commit fc345c1 into main Dec 25, 2024
1 check passed
@hee9841 hee9841 deleted the feature/#319/running-course branch December 25, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

러닝 저장 API v2 구현
2 participants